Skip to content

Conversation

mcarton
Copy link
Contributor

@mcarton mcarton commented Mar 3, 2016

It’s not even been merged yet, but rust-lang/rust#31797 will break the useless_vec lint.

@@ -71,7 +69,7 @@ impl LateLintPass for UselessVec {

/// Represent the pre-expansion arguments of a `vec!` invocation.
pub enum VecArgs<'a> {
/// `vec![elem, len]`
/// `vec![elem; len]`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That yak had a clean shave 😄

@llogiq
Copy link
Contributor

llogiq commented Mar 3, 2016

LGTM modulo next nightly.

mcarton added a commit that referenced this pull request Mar 4, 2016
Update to rustc 1.9.0-nightly (e91f889 2016-03-03)
@mcarton mcarton merged commit c0c2f61 into rust-lang:master Mar 4, 2016
@mcarton mcarton deleted the vec branch March 4, 2016 13:29
@mcarton
Copy link
Contributor Author

mcarton commented Mar 4, 2016

Ref rust-lang/rust#31979.

@mcarton mcarton changed the title vec! now uses box Update to rustc 1.9.0-nightly (e91f889ed 2016-03-03) Mar 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants